Skip to content

[stable33] fix(sharing): Prevent empty password when checkbox is enabled#59090

Open
backportbot[bot] wants to merge 3 commits intostable33from
backport/58226/stable33
Open

[stable33] fix(sharing): Prevent empty password when checkbox is enabled#59090
backportbot[bot] wants to merge 3 commits intostable33from
backport/58226/stable33

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 19, 2026

Backport of #58226

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot requested review from nfebe, susnux and szaimen March 19, 2026 19:24
@backportbot backportbot bot added bug 3. to review Waiting for reviews labels Mar 19, 2026
@backportbot backportbot bot added this to the Nextcloud 33.0.1 milestone Mar 19, 2026
Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@nfebe nfebe force-pushed the backport/58226/stable33 branch from a51ed15 to 4754104 Compare March 23, 2026 15:48
@nfebe
Copy link
Contributor

nfebe commented Mar 23, 2026

/backport to stable32

nfebe added 2 commits March 24, 2026 10:55
Set passwordProtectedState explicitly when initializing shares with
default passwords. This ensures the checkbox state is tracked
independently of the password value, preventing it from unchecking
when the password field is cleared.

Also block saving new shares when password protection is enabled but
no password is entered, regardless of enforcement settings.

Added passWithNoTests to vitest configs to handle Vue 2/3 dual
frontend test runs gracefully.

Fixes: #57732, #57011
Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe nfebe force-pushed the backport/58226/stable33 branch from 4754104 to 903d568 Compare March 24, 2026 09:56
@nfebe nfebe marked this pull request as ready for review March 24, 2026 09:57
@nfebe nfebe requested a review from a team as a code owner March 24, 2026 09:57
@nfebe nfebe requested review from skjnldsv and sorbaugh and removed request for a team March 24, 2026 09:57
@nfebe nfebe enabled auto-merge March 24, 2026 09:57
@nfebe
Copy link
Contributor

nfebe commented Mar 24, 2026

/compile /

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants